Skip to content

Fix downgrade.yml workflow: add missing version arguments#53

Merged
Krastanov merged 2 commits into
QuantumSavory:mainfrom
Krastanov-agent:fix-downgrade-workflow-version-args
Feb 8, 2026
Merged

Fix downgrade.yml workflow: add missing version arguments#53
Krastanov merged 2 commits into
QuantumSavory:mainfrom
Krastanov-agent:fix-downgrade-workflow-version-args

Conversation

@Krastanov-agent

Copy link
Copy Markdown
Contributor

Summary

The downgrade workflow requires proper version specification for both setup-julia and julia-downgrade-compat actions to work correctly.

Changes

This PR fixes the following issues in .github/workflows/downgrade.yml:

  • ✅ Ensures julia-actions/setup-julia has version: ${{ matrix.version }} argument
  • ✅ Ensures julia-actions/julia-downgrade-compat has julia_version: ${{ matrix.version }} argument
  • ✅ Updates action versions to v2 where needed

Why this matters

Without the julia_version argument, the downgrade compatibility action may not use the correct Julia version from the test matrix, leading to incorrect test results.

Test plan

  • Verify downgrade.yml has proper version arguments
  • Check that the workflow runs successfully after merge

Reference

Correct example: https://github.com/QuantumSavory/PBCCompiler.jl/blob/master/.github/workflows/downgrade.yml

🤖 Generated with Claude Code

Krastanov and others added 2 commits February 7, 2026 19:59
The downgrade workflow requires:
- `version` argument in setup-julia step
- `julia_version` argument in julia-downgrade-compat action

Without these, the workflow may fail or use incorrect Julia versions.

This PR ensures both arguments are properly specified to match the
matrix version.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Feb 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.49%. Comparing base (1214fd5) to head (7d5884e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #53      +/-   ##
==========================================
+ Coverage   95.11%   95.49%   +0.37%     
==========================================
  Files           7        7              
  Lines         757      754       -3     
==========================================
  Hits          720      720              
+ Misses         37       34       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Krastanov Krastanov merged commit 8481367 into QuantumSavory:main Feb 8, 2026
8 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants